Skip to content

coz: Causal profiler#3282

Closed
klimkin wants to merge 3 commits into
conan-io:masterfrom
klimkin:feature/add-coz
Closed

coz: Causal profiler#3282
klimkin wants to merge 3 commits into
conan-io:masterfrom
klimkin:feature/add-coz

Conversation

@klimkin
Copy link
Copy Markdown
Contributor

@klimkin klimkin commented Oct 20, 2020 •
edited
Loading

coz/0.2.2

Requires libelfin/0.3 #3283

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

@conan-center-bot
Copy link
Copy Markdown
Contributor

conan-center-bot commented Oct 20, 2020

config.yml syntax error in build 1:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@klimkin klimkin changed the title Add libelfin and coz coz: Causal profiler Oct 20, 2020
@conan-center-bot
Copy link
Copy Markdown
Contributor

conan-center-bot commented Oct 20, 2020

config.yml syntax error in build 2:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@uilianries
Copy link
Copy Markdown
Member

uilianries commented Oct 21, 2020

@klimkin Please, separate libelfin from this PR.

@conan-center-bot
Copy link
Copy Markdown
Contributor

conan-center-bot commented Oct 23, 2020

config.yml syntax error in build 3:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@klimkin
libelfin: reading ELF binaries and DWARFv4 debug information
f190114 
C++11 library for reading ELF binaries and DWARFv4 debug information.
@conan-center-bot
Copy link
Copy Markdown
Contributor

conan-center-bot commented Oct 23, 2020

config.yml syntax error in build 4:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@klimkin
Copy link
Copy Markdown
Contributor Author

klimkin commented Oct 23, 2020 •
edited
Loading

@klimkin Please, separate libelfin from this PR.

@uilianries This recipe requires libelfin. For libelfin, please see #3283

@uilianries
Copy link
Copy Markdown
Member

uilianries commented Oct 26, 2020

@klimkin This PR still contains 2 different packages. CCI does allow it, please, remove one of these packages.

@klimkin
Copy link
Copy Markdown
Contributor Author

klimkin commented Oct 26, 2020

@klimkin This PR still contains 2 different packages. CCI does allow it, please, remove one of these packages.

@uilianries coz requires libelfin which is submitted as a separate PR. This PR is rebased on #3283. It won't build is libelfin is not available.

@uilianries
Copy link
Copy Markdown
Member

uilianries commented Oct 28, 2020

@klimkin if it won't build, please close this PR.

@klimkin @SpaceIm
Apply suggestions from code review
3ad8018 
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
@conan-center-bot
Copy link
Copy Markdown
Contributor

conan-center-bot commented Oct 28, 2020

config.yml syntax error in build 5:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

This was referenced Oct 28, 2020
Closed
Closed
@klimkin
Copy link
Copy Markdown
Contributor Author

klimkin commented Oct 28, 2020

@emeryberger I noticed coz mentions coz-profiler as the package name, but the repo name is coz. Which name for the package would you prefer?

@klimkin
coz: Causal profiler
fbae9df 
Coz is a new kind of profiler that unlocks optimization opportunities
missed by traditional profilers. Coz employs a novel technique we call
causal profiling that measures optimization potential. This measurement
matches developers' assumptions about profilers: that optimizing
highly-ranked code will have the greatest impact on performance. Causal
profiling measures optimization potential for serial, parallel, and
asynchronous programs without instrumentation of special handling for
library calls and concurrency primitives. Instead, a causal profiler
uses performance experiments to predict the effect of optimizations.
This allows the profiler to establish causality: "optimizing function X
will have effect Y," exactly the measurement developers had assumed they
were getting all along.

https://github.com/plasma-umass/coz
@conan-center-bot
Copy link
Copy Markdown
Contributor

conan-center-bot commented Oct 28, 2020

config.yml syntax error in build 6:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@emeryberger
Copy link
Copy Markdown

emeryberger commented Oct 30, 2020

Hi - I'd prefer coz if that's possible!

@ericLemanissier ericLemanissier mentioned this pull request Nov 12, 2020
Closed
@ghost ghost mentioned this pull request Nov 16, 2020
Closed
@stale
Copy link
Copy Markdown

stale Bot commented Dec 1, 2020

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the stale label Dec 1, 2020
@klimkin
Copy link
Copy Markdown
Contributor Author

klimkin commented Dec 3, 2020

Will resubmit once #3283 is merged.

@klimkin klimkin closed this Dec 3, 2020
@emeryberger
Copy link
Copy Markdown

emeryberger commented Jan 14, 2021

Noticed that #3283 was merged, FWIW.

@klimkin
Copy link
Copy Markdown
Contributor Author

klimkin commented Jan 14, 2021

@emeryberger See re-opened #3794. This is now waiting for merging CMake build for coz, since it will simplify the recipe: plasma-umass/coz#178

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Failed stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@klimkin @conan-center-bot @uilianries @emeryberger